Project

General

Profile

Actions

Feature #545

open

Split flashchips.c into smaller files, one per manufacturer

Added by Anastasia Klimchuk 5 months ago. Updated 5 months ago.

Status:
New
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

Also available in: Atom PDF