Project

General

Profile

Actions

Feature #545

closed

Split flashchips.c into smaller files, one per manufacturer

Added by Anastasia Klimchuk 6 months ago. Updated 10 days ago.

Status:
Resolved
Priority:
Normal
Category:
-
Target version:
Start date:
06/27/2024
Due date:
% Done:

0%

Estimated time:
Affected versions:
Needs backport to:
Affected hardware:
Affected OS:

Description

From meeting notes on 27/06/2024

How about, we split our large file flashchips.c into smaller files, per manufacturer?

  • seems like a good idea
  • smaller files can get different maintainers per file => easier to get attention to your patch for flashchip
  • less merge conflicts
  • easier to find chips, easier to maintain

For example,

const struct flashchip flashchips[] = {
#include "chips/gigadevice.inc"
#include "chips/macronix.inc"
#include "chips/winbond.inc"
}
.inc is not valid c file, just a piece of code (slice array)

another idea can be array of .c files

Actions #1

Updated by Anastasia Klimchuk 6 months ago

  • Assignee set to Anastasia Klimchuk
Actions #2

Updated by Anastasia Klimchuk 10 days ago

  • Status changed from New to Resolved
Actions

Also available in: Atom PDF